Skip to content

docs: fix broken Open Policy Agent link in multi-tenant guide#1062

Open
kruthiwusirika5 wants to merge 1 commit intocrossplane:masterfrom
kruthiwusirika5:fix-multi-tenant-open-policy-agent-link
Open

docs: fix broken Open Policy Agent link in multi-tenant guide#1062
kruthiwusirika5 wants to merge 1 commit intocrossplane:masterfrom
kruthiwusirika5:fix-multi-tenant-open-policy-agent-link

Conversation

@kruthiwusirika5
Copy link
Copy Markdown

@kruthiwusirika5 kruthiwusirika5 commented Feb 23, 2026

Remove space between ] and ( in markdown link so reference-style link [Open Policy Agent] resolves correctly to the URL defined at the bottom of the file.

@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 23, 2026 •
edited
Loading

✅ Deploy Preview for crossplane ready!

Name Link
🔨 Latest commit c15791a
🔍 Latest deploy log https://app.netlify.com/projects/crossplane/deploys/699bb9708a6d79000873a813
😎 Deploy Preview https://deploy-preview-1062--crossplane.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse		 1 paths audited
Performance: 93 (🔴 down 3 from production)
Accessibility: 90 (🔴 down 2 from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: 70 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@kruthiwusirika5 kruthiwusirika5 force-pushed the fix-multi-tenant-open-policy-agent-link branch from 327ab05 to 5b030bf Compare February 23, 2026 02:10
@kruthiwusirika5
docs: fix broken Open Policy Agent link in multi-tenant guide
c15791a 
Remove space between ] and ( in markdown link so reference-style
link [Open Policy Agent] resolves correctly to the URL defined
at the bottom of the file.

Signed-off-by: Kruthi Wusirika <kruthiwusirika@gmail.com>
@kruthiwusirika5 kruthiwusirika5 force-pushed the fix-multi-tenant-open-policy-agent-link branch from 5b030bf to c15791a Compare February 23, 2026 02:20
@kruthiwusirika5
Copy link
Copy Markdown
Author

kruthiwusirika5 commented Mar 16, 2026

The lint-content check failure is pre-existing in content/v1.20/guides/multi-tenant.md and was not introduced by this PR. The Vale run reports 0 errors and 82 warnings — all of which are existing style issues in the file (passive voice, word substitutions, heading capitalization, etc.) unrelated to this change.
This PR only removes an errant space in a markdown link to fix the Open Policy Agent reference. No new prose was added that could trigger Vale rules.
Happy to address the pre-existing Vale warnings in a separate follow-up PR if that would be helpful, but I'd prefer to keep this fix scoped. @haarchri — would you be able to merge given the above?

@kruthiwusirika5
Copy link
Copy Markdown
Author

kruthiwusirika5 commented Apr 30, 2026

@haarchri — gentle nudge on this one-line fix (already approved 2026-03-13). Just removes a stray space in [Open Policy Agent] (OPA) so the reference-style link resolves to its URL definition further down the file.

The only red CI is the Vale lint-content job, which reports 0 errors, 82 warnings — all pre-existing style issues in content/v1.20/guides/multi-tenant.md (passive voice, word substitutions, heading capitalization, etc.) that this PR does not touch. No new prose was added that could trigger Vale.

If the warnings are a hard merge gate, happy to open a separate PR addressing them. If they're not, this should be safe to land. Thanks!

jbw976
jbw976 reviewed Apr 30, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@jbw976 jbw976 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for following up on this! I was just checking the live page now and the link seems to resolve and work for me OK. are you still seeing this issue on the live page?

is this the right paragraph? https://docs.crossplane.io/v1.20/guides/multi-tenant/#policy-enforcement-with-open-policy-agent

@kruthiwusirika5
Copy link
Copy Markdown
Author

kruthiwusirika5 commented Apr 30, 2026

@jbw976 — thanks for double-checking. You're right: just verified that
the live page renders correctly. I'd misdiagnosed this — Hugo's CommonMark
renderer treats [Open Policy Agent] (OPA) as a valid shortcut reference
link followed by literal (OPA) text, not as an inline link with a
malformed URL.

So the change here is cosmetic, not a fix: it just removes the redundant
(OPA) after the resolved link. Keeping the acronym is also defensible
(spells out OPA on first mention). Happy to close this PR — sorry for
the mis-framing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jbw976 jbw976 jbw976 left review comments

@haarchri haarchri haarchri approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kruthiwusirika5 @jbw976 @haarchri